Skip to content

Conversation

@mevionfamos
Copy link
Contributor

@mevionfamos mevionfamos commented Jan 16, 2026

Description

This PR adds the theme tooltip zindex to the share tooltip to make sure it is visible on mobile.

Related Issues

fixes #553

Testing Performed

  • Tested with the following Browsers: Safari, Brave
  • Tested on the following devices: iPhone, Macbook
  • Verified functionality: share "works"
  • Storybook updated
  • Automated tests added

Screenshots

image

Checklist

  • I have tested my changes thoroughly
  • I have updated the documentation as needed
  • My commits use clear, descriptive messages
  • My PR includes only related changes
  • I have marked this PR with the appropriate label
  • I have added an entry in the changelog
  • I have run locales:extract if I changed any locale string

Notes for Reviewers

@Muchete the tooltip now "works" (you can use it, it's not broken) but the width is fixed at 300px, and it is not super pretty. Im fine to leave it for now but let's think about a better design for this (e.g. full width as the details card)

@ptbrowne I didn't find a simple (1-2 lines of codes) way to achieve the same tooltip style on mobile as the details card so I didn't touch anything beyond the zindex for now. Feel free to improve the layout if you have time and take it from here. Note that at the moment, on very small screens (iPhone 13 mini) the tooltip causes a slight layout shift, which is not ideal. This is something that affects all tooltips and documented here: #555

@vercel
Copy link

vercel bot commented Jan 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
elcom-electricity-price-website Ready Ready Preview, Comment Jan 16, 2026 4:55pm

Request Review

@mevionfamos mevionfamos marked this pull request as draft January 16, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Mobile: Sharing drawing order

2 participants